Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add script to create auto payload #9

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add script to create auto payload #9

wants to merge 3 commits into from

Conversation

Maltmark
Copy link
Contributor

This script should run locally in the forked repo of BDG/aave-proposal
what it does:

  1. get config of requested assets changes, the supported changes:
    a. general params - support 1 line change, currently for caps (borrow and supply) and borrowable isolation. -easy to add more supported changes.
    b. risk params - support risk parma change(LTV,LT and LB)
  2. create 1 contract payload and 1 test contract according to config.
  3. run the test
  4. create a commit
  5. push changes to PR



class CreateFile(ABC):
def __init__(self, pr_name: str, paylaod_name: str, info: str, snapshot: str, discussion: str, config):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we define type for config, even TypeDict


return data_set_up

def get_all_params_data(self, asset_change_data, asset_name, is_test):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can u add types?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants